Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dev windows support part2 #243

Closed

Conversation

loongs-zhang
Copy link
Contributor

@loongs-zhang loongs-zhang commented Feb 20, 2024

This PR mainly uses polling as the LegacyDriver, as you can see, it can be successfully compiled and run a small number of tests.

Plan to pass really all windows CI in the next PR.

@loongs-zhang loongs-zhang force-pushed the dev-windows-support-part2 branch from 98d89f1 to b124657 Compare February 20, 2024 14:05
@loongs-zhang loongs-zhang force-pushed the dev-windows-support-part2 branch from 22be20a to 4a4a2c4 Compare February 20, 2024 14:33
#[cfg(windows)]
poll: iocp::Poller,
poll: Poller,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have 2 questions:

  1. Which way do you want to iocp? It can be used as epoll to just get notified if the fd is ready, it can also be used like iouring to let it do io directly.
  2. If it is designed to work as a readiness poller, can we use mio::Poll instead of brings a new poller impl?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Which way do you want to iocp? It can be used as epoll to just get notified if the fd is ready, it can also be used like iouring to let it do io directly.

Just epoll.

  1. If it is designed to work as a readiness poller, can we use mio::Poll instead of brings a new poller impl?

No, cause #166 (comment)

@loongs-zhang loongs-zhang reopened this Feb 21, 2024
@loongs-zhang loongs-zhang force-pushed the dev-windows-support-part2 branch 2 times, most recently from 0b98ba3 to 52ff11b Compare February 22, 2024 13:51
@loongs-zhang loongs-zhang reopened this Feb 22, 2024
@loongs-zhang loongs-zhang force-pushed the dev-windows-support-part2 branch from 27bbfd3 to f1182e6 Compare February 22, 2024 14:28
@loongs-zhang
Copy link
Contributor Author

It seems that due to caching, windows CI is unstable, see https://github.com/loongs-zhang/monoio/actions/runs/8005626446/job/21865508898

@loongs-zhang loongs-zhang marked this pull request as draft February 23, 2024 02:56
@loongs-zhang loongs-zhang mentioned this pull request Feb 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants